Skip to content

python3Packages.sphinxcontrib-mermaid: init at 1.0.0 #457940

Closed
adamecooper wants to merge 2 commits intoNixOS:masterfrom
adamecooper:add-sphinxcontrib-mermaid
Closed

python3Packages.sphinxcontrib-mermaid: init at 1.0.0 #457940
adamecooper wants to merge 2 commits intoNixOS:masterfrom
adamecooper:add-sphinxcontrib-mermaid

Conversation

@adamecooper
Copy link

Things done

  • Built on platform:
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • Tested, as applicable:
  • Ran nixpkgs-review on this PR. See nixpkgs-review usage.
  • Tested basic functionality of all binary files, usually in ./result/bin/.
  • Nixpkgs Release Notes
    • Package update: when the change is major or breaking.
  • NixOS Release Notes
    • Module addition: when adding a new NixOS module.
    • Module update: when the change is significant.
  • Fits CONTRIBUTING.md, pkgs/README.md, maintainers/README.md and other READMEs.

Add a 👍 reaction to pull requests you find important.

@adamecooper adamecooper changed the title Add sphinxcontrib mermaid Add sphinxcontrib-mermaid Nov 3, 2025
@nix-owners nix-owners bot requested a review from natsukium November 3, 2025 00:17
@nixpkgs-ci nixpkgs-ci bot added 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 12.first-time contribution This PR is the author's first one; please be gentle! 6.topic: python Python is a high-level, general-purpose programming language. 8.has: maintainer-list (update) This PR changes `maintainers/maintainer-list.nix` labels Nov 3, 2025
@sikmir sikmir changed the title Add sphinxcontrib-mermaid python3Packages.sphinxcontrib-mermaid: init at 1.0.0 Nov 3, 2025
@sikmir
Copy link
Member

sikmir commented Nov 3, 2025

--- i/pkgs/development/python-modules/sphinxcontrib-mermaid/default.nix
+++ w/pkgs/development/python-modules/sphinxcontrib-mermaid/default.nix
@@ -7,7 +7,7 @@
   setuptools,
   defusedxml,
   myst-parser,
-  pytest,
+  pytestCheckHook,
   ruff,
 }:

@@ -34,12 +34,13 @@ buildPythonPackage rec {
     test = [
       defusedxml
       myst-parser
-      pytest
       ruff
       sphinx
     ];
   };

+  nativeCheckInputs = [ pytestCheckHook ] ++ optional-dependencies.test;
+
   pythonImportsCheck = [ "sphinxcontrib.mermaid" ];

   meta = {

@adamecooper adamecooper force-pushed the add-sphinxcontrib-mermaid branch from 686da35 to 9a17cbd Compare November 3, 2025 14:29
@nixpkgs-ci nixpkgs-ci bot added the 2.status: merge conflict This PR has merge conflicts with the target branch label Nov 28, 2025
@kirillrdy
Copy link
Member

superseded by #463822

@adamecooper adamecooper closed this Dec 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

2.status: merge conflict This PR has merge conflicts with the target branch 6.topic: python Python is a high-level, general-purpose programming language. 8.has: maintainer-list (update) This PR changes `maintainers/maintainer-list.nix` 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 12.first-time contribution This PR is the author's first one; please be gentle!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants